Skip to content

DataModel: improve DataHeader formatter#14911

Closed
ktf wants to merge 4 commits intoAliceO2Group:devfrom
ktf:pr14911
Closed

DataModel: improve DataHeader formatter#14911
ktf wants to merge 4 commits intoAliceO2Group:devfrom
ktf:pr14911

Conversation

@ktf
Copy link
Member

@ktf ktf commented Dec 11, 2025 •
edited
Loading

Add splitPayloadIndex / splitPayloadParts to the default printout

Stack created with Sapling. Best reviewed with ReviewStack.

ktf added 4 commits December 10, 2025 09:54
@ktf
DPL: more refactoring of the forwarding code
d54ec0d 
Use a single helper function to improve readability.
@ktf
DPL: fix routing issues in forwarding
135238e 
If one (header, payload, ...) tuple in a MessageSet was to be copied,
all the subsequent ones would have been copied.

If one (header, payload, ...) tuple got redirected to more than one destination,
all the subsequent ones would have been redirected there.
@ktf
DPL: avoid MessageSet abstractions when forwarding
1ce6469 
This is most likely faster, and it will allow us to move
the early forwarding at an earlier stage where the data is not
yet in a MessageSet.
@ktf
DataModel: improve DataHeader formatter
c9e9cc8 
Add splitPayloadIndex / splitPayloadParts to the default printout
@ktf ktf requested a review from a team as a code owner December 11, 2025 09:38
This was referenced Dec 11, 2025
Merged
Merged
@github-actions
Copy link
Contributor

github-actions bot commented Dec 11, 2025

REQUEST FOR PRODUCTION RELEASES:
To request your PR to be included in production software, please add the corresponding labels called "async-" to your PR. Add the labels directly (if you have the permissions) or add a comment of the form (note that labels are separated by a ",")

+async-label <label1>, <label2>, !<label3> ...

This will add <label1> and <label2> and removes <label3>.

The following labels are available
async-2023-pbpb-apass4
async-2023-pp-apass4
async-2024-pp-apass1
async-2022-pp-apass7
async-2024-pp-cpass0
async-2024-PbPb-apass1
async-2024-ppRef-apass1
async-2024-PbPb-apass2
async-2023-PbPb-apass5

@alibuild
Copy link
Collaborator

alibuild commented Dec 11, 2025

Error while checking build/O2/fullCI_slc9 for c9e9cc8 at 2025-12-11 17:46:

## sw/BUILD/O2-full-system-test-latest/log
command o2-sim --field=ccdb --seed 12345 -j 32 -n 100 -m PIPE ITS MFT FT0 FV0 FDD -g extgen -e TGeant4 --configKeyValues "GeneratorExternal.fileName=$O2_ROOT/share/Generators/external/QEDLoader.C;QEDGenParam.yMin=-7;QEDGenParam.yMax=7;QEDGenParam.ptMin=0.001;QEDGenParam.ptMax=1.;${SIMOPTKEY}" --run 310000 had nonzero exit code 139


## sw/BUILD/o2checkcode-latest/log
--
========== List of errors found ==========
++ GRERR=0
++ grep -v clang-diagnostic-error error-log.txt
++ grep ' error:'
++ GRERR=1
++ [[ 1 == 0 ]]
++ mkdir -p /sw/INSTALLROOT/55ced40990c08395a68ba6a083bc1eeb30bc6861/slc9_x86-64/o2checkcode/1.0-local41/etc/modulefiles
++ cat
--

Full log here.

@ktf ktf closed this Jan 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ktf @alibuild